Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PISTON-1002: use collect_digits for DTMF during vm greeting/instructions #6249

Conversation

danielfinke
Copy link
Member

When play_greeting_intro, play_greeting, or play_instructions play media, a noop is enqueued. kapps_call_command:wait_for_application_or_dtmf may continue on the noop from one of these play commands, rather than as expected due to kapps_call_command:noop. This means, for example, the user is unable to press a key to connect to the operator during instructions, if the greeting has already been played.

I introduced the 8-arity collect_digits for this purpose, too, as it allows a long initial timeout (kapps_call_command:default_application_timeout) but continues immediately after the expecting noop (the second argument set to 0).

@lazedo lazedo closed this Feb 27, 2020
@lazedo lazedo reopened this Feb 27, 2020
@lazedo
Copy link
Member

lazedo commented May 1, 2020

@danielfinke rebase here

@danielfinke danielfinke force-pushed the pr-PISTON-1002-voicemail-operator-number-during-instructions branch from 738c325 to da455f0 Compare May 1, 2020 16:34
@danielfinke danielfinke force-pushed the pr-PISTON-1002-voicemail-operator-number-during-instructions branch from da455f0 to 1c01c04 Compare May 1, 2020 17:34
@lazedo lazedo merged commit 2d45108 into 2600hz:master May 1, 2020
@danielfinke danielfinke deleted the pr-PISTON-1002-voicemail-operator-number-during-instructions branch May 1, 2020 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants